Skip to content

feat: season standing over the rounds of a division, checked against six more official races - #2

Merged
malpou merged 1 commit into
mainfrom
feat/season-standing
Oct 19, 2026
Merged

malpou merged 1 commit into
mainfrom
feat/season-standing

Conversation

@malpou

@malpou malpou commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Rulebook.Season ranks a division over its rounds. It adds the match points and the scores of all rounds, and the tie-breaks read the matches and groups of all rounds. Clubs are matched by name, so the caller gives a club one name for the whole season.
  • Six more golden races: 1st and 2nd division Øst, round 1 and round 2; 1st, 2nd and 3rd division Nord, round 1 and round 2; and the DM Hold final with 6 clubs and 15 matches.
  • The golden test now also checks the season standing at the top of each printout: every round's numbers, the totals and the tie-break comment. A round-2 race names round 1 with before. A scanned printout is a hand transcription of its summary pages, marked transcribed: true, without runner rows.

Evidence

go test -v -run Golden . agrees on all 14 divisions of the 7 races:

  • every standing line and match score;
  • every cell of the group-by-match grid;
  • 1,241 more runner rows from the text printouts;
  • every season line, including the tie-break comments such as 16 vs 14 first places in DM Hold and the mutual scores over two rounds in 3rd division Nord.

All runner names are generic. golangci-lint has 0 issues.

Merge Danger

Door: two-way. New API and test data; Compute is unchanged.
Blast Radius: small.

@malpou
malpou merged commit 63dbe71 into main Oct 19, 2026
2 checks passed
@malpou
malpou deleted the feat/season-standing branch October 19, 2026 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant